Finding the owner of an Weapon correctly #340
Conversation
Astralcircle
changed the title
There was a problem hiding this comment.
Thanks for the PR!
I did this because some addons like Wiremod use CanTool for CPPI checks. But since we can't interact with our weapon, some functions stop working (like weapon core).
I'm not sure I understand the problem you're describing. What are weapon core functions and what does "stop working" entail specifically?
The problem is that the player doesn't own his weapon. We also get :DropWeapon() support to know who dropped it |
|
Why was this closed? |
|
I thought that this is not what should be added. |
|
Oh, I think I may have come across differently than I intended. Sorry about that. My questions were asking for clarification on what problem it solves exactly. You may indeed be right in that this is a good solution. So as I understand it now, the CPPIOwner is not set for carried weapons. I guess some addons don't realize that these entities are carried weapons (e.g. E2) , so instead of using |
|
Yes. That's what I'm trying to get across. |
We already do this for weapons created by toolgun, but not for those given out via Q-menu or other means. I did this because some addons like Wiremod use CanTool for CPPI checks. But since we can't interact with our weapon, some functions stop working (like weapon core).
It might be worth overwriting the weapon owner to always match :GetOwner(), but I'm not sure if that's what you want to do, so I copied the PlayerSpawnedSWEP behavior